Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Collaborator
Author
This stack of pull requests is managed by Graphite. Learn more about stacking. |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
| - name: Install dependencies | ||
| run: pnpm install --frozen-lockfile | ||
|
|
||
| - name: Install Doppler CLI |
There was a problem hiding this comment.
Removing Doppler auth breaks CI unit tests
High Severity
The Doppler CLI installation and authentication steps were removed from the test job, but the pnpm test command still runs and will invoke test scripts in multiple packages (fmodata, better-auth, cli, fmdapi) that use doppler run -- to inject secrets. Without Doppler authentication, these test commands will fail and break the CI workflow. The Doppler removal was intended to support removing E2E tests, but unit tests in these packages also depend on Doppler.
Remove fmodata E2E step and Doppler auth from continuous-release.yml. Add root-level test:e2e script and turbo task for local E2E runs. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
eluce2
force-pushed
the
01-12-ci_remove_e2e_tests_from_ci_workflow
branch
from
e004b71 to
e1d1214
Compare
3 tasks
Collaborator
Author
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Remove fmodata E2E step and Doppler auth from continuous-release.yml.
Add root-level test:e2e script and turbo task for local E2E runs.
Co-Authored-By: Claude Opus 4.5 noreply@anthropic.com
Note
Streamlines CI and enables local E2E runs.
@proofkit/fmodataE2E step fromcontinuous-release.yml; CI now runs lint, typecheck, unit tests, and build onlytest:e2escript and a non-cachedtest:e2eturbo task for local E2E executionWritten by Cursor Bugbot for commit e1d1214. This will update automatically on new commits. Configure here.